feat: new ui home screen and receive screen #2983
Conversation
There was a problem hiding this comment.
Hey, tested PR with existing account, a few minor fixed are needed:
- delete shadow from card-buttons
- add 1px border to card-buttons
- bitcoin icon on "Bitcoin address" card needs to be swapped for those from Popicons
- could you add 2px spacing between title and paragraph in the card?
|
@stackingsaunter using border instead of shadow there leading to card taking more space and cutting text into new line
|
|
maybe we can use 12px item spacing instead of 16 px |
There was a problem hiding this comment.
Currently the two cards for the mnemonic are always being displayed:
Also after backing up or importing it, which IMO is not shipable in it's current form.
What about we introduce a backupDone property that is being set when confirming the backups / importing an external mnemonic?
|
added. functionality to add new property isMnemonicBackupDone and use it to show card when it's relevent to show |
|
for backup screen i will just set the property to true when new ui for mnemonic onboarding is merged. currently it is set only in generateScreen |
i reduced item-sacing from 16 to 12 px to resolve above |
|
@pavanjoshi914 Please fix the tests. |
Describe the changes you have made in this PR
new ui for home screen and receive screen
Link this PR to an issue [optional]
Fixes #2922
Type of change
(Remove other not matching type)
feat: New feature (non-breaking change which adds functionality)Checklist